PR para correção e comentários #7
Conversation
Atualização do index.html e do index.css
Nosso Time e Quem Somos
…otime.html' finalizadas (incluindo responsividade).
Ajuste do 'index.css'
Inclusão do arquivo novos-produtos.html
Inclusão do arquivo novos-produtos.html
Ajuste do layout da página 'nossos-produtos.html'
Teste para Deploy
paula-aribeiro
left a comment
paula-aribeiro
left a comment
There was a problem hiding this comment.
Oi pessoal, parabéns pelo trabalho de vocês!!!
Em relação ao código, tenho nem o que dizer, está impecável 😍 super organizado, com as tags semânticas do HTML5, muito show mesmo. Tem uma ou outra coisinha pra arrumar nos comentários.
Em relação ao layout, eu só destacaria a escolha de fonte. Entendo que vocês quiseram seguir a tipografia do logo e isso foi bem legal, mas acho que vocês poderiam ter escolhido uma fonte mais simples para os textos maiores. Isso facilita a leitura para o usuário.
Parabéns por terem criado páginas bem completas, por terem feito a responsividade e por terem sido um dos poucos grupos a fazer o deploy 👏
| </head> | ||
| <body> | ||
| <header> | ||
| <a href="index.html"><img id="logo-bank" src="f4bank.png" alt="logo"></a> <nav> |
There was a problem hiding this comment.
Faltou pular uma linha antes do <nav>
|
|
||
|
|
There was a problem hiding this comment.
Cuidado com linhas em branco sem necessidade
|
|
||
|
|
||
| /* Responsividade */ | ||
| @media screen and (max-device-width : 480px) { |
5 participants
No description provided.